Skip to content

Release v1.4.0 - Clarity Integration & Architecture Improvements #3

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
moreirawebmaster merged 1 commit into from
Jul 2, 2025
Merged

Release v1.4.0 - Clarity Integration & Architecture Improvements #3

moreirawebmaster merged 1 commit into from
Jul 2, 2025

Conversation

@moreirawebmaster
Copy link
Owner

@moreirawebmaster moreirawebmaster commented Jul 2, 2025

Release v1.4.0 - Clarity Integration & Architecture Improvements

🚀 New Features

  • Direct Clarity Integration: Removed adapter pattern for Microsoft Clarity, now integrated directly via EngineWidget
  • STMR Logo Integration: Added official STMR logo to assets and README.md
  • Enhanced README: Complete restructure with 7 organized sections including mind maps

🔧 Technical Improvements

  • Modern Lint Rules: Updated analysis_options.yaml to flutter_lints 6.0.0 with 350+ strict rules for Dart/Flutter 2025
  • New Widget Components: Added EngineWidget, EngineMaskWidget and EngineClarityConfig
  • Architecture Cleanup: Removed unnecessary clarity adapter while maintaining configuration support

📚 Documentation

  • Comprehensive Mind Maps: 5 Mermaid diagrams showing complete architecture
  • Clean Structure: Moved development rules to .cursor/rules.md, keeping README focused on functionality
  • Enhanced Examples: Updated usage examples and contribution guidelines

🏢 Brand Updates

  • STMR Presence: Added official logo and expanded company description
  • Professional Presentation: Centralized badges and improved visual hierarchy

🔒 Code Quality

  • Strict Standards: Implemented security, performance and modern Dart features
  • Zero Comments Policy: 100% self-explanatory code
  • Comprehensive Testing: All existing tests maintained and passing

Ready for production deployment! 🎉

@moreirawebmaster
feat: add v1.4.0 release with clarity integration
40ca119 
Update README.md, add STMR logo, modernize lint rules
@moreirawebmaster moreirawebmaster merged commit 89dd9f3 into main Jul 2, 2025
4 checks passed
@moreirawebmaster moreirawebmaster deleted the release/v1.4.0 branch July 2, 2025 16:27
cursor[bot]
cursor bot reviewed Jul 2, 2025
View reviewed changes
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: Reset Method Missing Data Field

The reset() method omits the 'data' field from its _sendToSplunk payload, unlike other methods (e.g., logEvent, setUserId, setUserProperty, setPage, logAppOpen) which include it. This inconsistency may cause issues if the Splunk backend expects all events to contain a data field.

lib/src/analytics/adapters/engine_splunk_analytics_adapter.dart#L161-L165

engine-tracking/lib/src/analytics/adapters/engine_splunk_analytics_adapter.dart

Lines 161 to 165 in 40ca119

try {
await _sendToSplunk({
'event': 'analytics_reset',
'timestamp': DateTime.now().millisecondsSinceEpoch / 1000,
});

Fix in Cursor

BugBot free trial expires on July 22, 2025
You have used $0.00 of your $50.00 spend limit so far. Manage your spend limit in the Cursor dashboard.

Was this report helpful? Give feedback by reacting with 👍 or 👎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cursor cursor[bot] cursor[bot] left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@moreirawebmaster